Skip to content

Demo improvements - #1

Merged
sl5-guy merged 4 commits into
mainfrom
demo-improvements
May 8, 2026
Merged

Demo improvements#1
sl5-guy merged 4 commits into
mainfrom
demo-improvements

Conversation

@Peter-SL5

Copy link
Copy Markdown
Contributor

No description provided.

@Peter-SL5
Peter-SL5 force-pushed the demo-improvements branch from e08a1c7 to 8ecefc8 Compare May 7, 2026 22:42
Comment thread demo/setup.sh Outdated
fi
# Ensure the gateway (running in Docker) sends OTLP to the collector service name,
# not localhost (which would be the gateway container itself).
sed -i 's|otlp_endpoint = "http://localhost:4317"|otlp_endpoint = "http://otel-collector:4317"|' "$REPO_ROOT/config.toml"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a regression

Comment thread demo/demo.sh
@Peter-SL5
Peter-SL5 force-pushed the demo-improvements branch 2 times, most recently from 4918130 to e77f41b Compare May 8, 2026 00:00
Comment thread demo/demo.sh
TPM2_PKCS11_STORE="${AGENT_GATEWAY_DEMO_TPM2_PKCS11_STORE:-$STATE_ROOT/tpm2-pkcs11}"
USER_PIN="${AGENT_GATEWAY_TPM_USER_PIN:-agentgateway}"
SO_PIN="${AGENT_GATEWAY_TPM_SO_PIN:-agentgateway-so}"
DATABASE_URL="${AGENT_GATEWAY_DATABASE_URL:-postgres://agent_gateway_admin:agent_gateway_dev@127.0.0.1:5432/agent_gateway}"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you reword the commit to refer to the new file name?

@Peter-SL5
Peter-SL5 force-pushed the demo-improvements branch from e77f41b to f4d2d34 Compare May 8, 2026 00:28
@sl5-guy
sl5-guy merged commit 30d7575 into main May 8, 2026
@luiscosio
luiscosio deleted the demo-improvements branch July 10, 2026 18:54
luiscosio added a commit that referenced this pull request Jul 24, 2026
* ⬆️ chore(deps): upgrade all dependencies to latest majors

Bump every dependency to its latest released major. 16 of 21 direct deps
were already on the latest major; 5 needed incompatible bumps:

- opentelemetry        0.31 -> 0.32
- opentelemetry_sdk    0.31 -> 0.32  (clears Dependabot alert #1)
- opentelemetry-otlp   0.31 -> 0.32
- tracing-opentelemetry 0.32 -> 0.33
- sha2                 0.10 -> 0.11

No source changes were required. The observability setup already used the
post-0.30 opentelemetry API (SdkTracerProvider, Resource::builder_empty,
with_batch_exporter), which 0.32 kept stable, and the sha2 0.11 bump keeps
the Digest/Sha256 trait surface used in identity.rs.

cargo update also pulled transitive fixes:

- rustls-webpki 0.103.11 -> 0.103.13  (RUSTSEC-2026-0098/0099/0104)
- anyhow        1.0.102  -> 1.0.104   (RUSTSEC-2026-0190 unsoundness)

cargo audit goes from 3 vulnerabilities to 0.

Verified on aarch64-linux (tss-esapi-sys has no aarch64-darwin bindings):
cargo build, cargo build --release --locked, and cargo test all pass,
24/24 tests green including the swtpm-backed TPM signing test.

* 🧹 fix(sidecar): use infallible KeyHandle conversion

`ObjectHandle` -> `KeyHandle` cannot fail, so `try_from` plus an
`anyhow::Context` wrapper was dead error handling. clippy's
`unnecessary_fallible_conversions` lint made this a hard error under
`-D warnings`, which this crate escalates further via `pedantic = "deny"`.

Pre-existing on main; left out of the dependency commit to keep that
diff scoped to dependencies.

Verified on aarch64-linux (rust:1.94-trixie with libtss2-dev, swtpm,
tpm2-tools) because tss-esapi-sys ships no aarch64-darwin bindings:
  cargo clippy --all-targets -- -D warnings   exit 0
  cargo test                                  24 passed, 0 failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants